Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

todo.txt: Fix trailing space resulting in contexts/projects/due-dates to be entered twice #1282

Merged
merged 2 commits into from
Mar 14, 2021
Merged

todo.txt: Fix trailing space resulting in contexts/projects/due-dates to be entered twice #1282

merged 2 commits into from
Mar 14, 2021

Conversation

harshad1
Copy link
Collaborator

i.e.

foo_  <- insert due date

results in

foo due:2021-03-13 due:2021-03-13

(_ = space)

@harshad1
Copy link
Collaborator Author

This is caused as \s*$ matches a trailing space 2x for some reason

@gsantner this may be worth a bugfix release

@gsantner gsantner added the bug label Mar 14, 2021
@gsantner gsantner added this to the v2.7 milestone Mar 14, 2021
@gsantner gsantner linked an issue Mar 14, 2021 that may be closed by this pull request
Due date 'days in future' should be configurable #1093
Closed
@gsantner gsantner changed the title BUGFIX: Trailing space causes todo.txt contexts/projects/due-dates to be entered twice todo.txt: Fix trailing space resulting in contexts/projects/due-dates to be entered twice Mar 14, 2021
@gsantner gsantner merged commit ef0ef5c into gsantner:master Mar 14, 2021
@harshad1 harshad1 deleted the fix_due_date_trailing_space branch April 11, 2022 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@harshad1 harshad1

Labels
bug
Projects
None yet
Milestone
v2.7
Development

Successfully merging this pull request may close these issues.

Due date 'days in future' should be configurable
2 participants
@harshad1 @gsantner